Skip to content

Conversation

@siddharth1036
Copy link
Contributor

Description

This commit adds indexing for referencegrants. The indexing key uses To Kind + namespace in referencegrant for indexing which would help fetch only related referencegrants given aiservicebackend and namespace.

Related Issues/PRs (if applicable)
Closes #1375

Signed-off-by: siddharth1036 <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.24%. Comparing base (a2ba2c9) to head (6e3b0de).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1618      +/-   ##
==========================================
- Coverage   83.26%   83.24%   -0.03%     
==========================================
  Files         137      137              
  Lines       12059    12389     +330     
==========================================
+ Hits        10041    10313     +272     
- Misses       1411     1446      +35     
- Partials      607      630      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: siddharth1036 <[email protected]>
@siddharth1036 siddharth1036 marked this pull request as ready for review December 5, 2025 10:52
@siddharth1036 siddharth1036 requested a review from a team as a code owner December 5, 2025 10:52
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 5, 2025
}

func getReferenceGrantIndexKey(namespace, kind string) string {
return fmt.Sprintf("%s/%s", namespace, kind)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you use the same format as others (name.namespace)

Signed-off-by: siddharth1036 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Indexing Support for reference grant resource

3 participants